Skip to content

Adapts partially to Py3. Adds a known good npm/nodejs download channel. #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
loriab wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adapts partially to Py3. Adds a known good npm/nodejs download channel. #46

loriab wants to merge 1 commit into from

Conversation

@loriab
Copy link

@loriab loriab commented Jun 9, 2017

Far from comprehensive, but catches a few Py3 problems I saw in trying to run Tutorial 3. Also specifies conda-forge nodejs package as a worthy way to get "npm", rather than the broken npm conda package on cpcloud channel.

I notice afterward your py3 branch, so let me know if you'd rather have this PR to that.

Current error for Tutorial 3 is:

... last 1 frames repeated, from the frame below ...

~/linux/notebook-molecular-visualization/nbmolviz/widgets/selector.py in __getattr__(self, item)
    105 
    106     def __getattr__(self, item):
--> 107         if self.viewer is not None: return getattr(self.viewer, item)
    108         else: raise AttributeError(item)
    109 

RecursionError: maximum recursion depth exceeded while calling a Python object

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@loriab